Skip to content

committing sho#22

Open
Swaraj26 wants to merge 1 commit intotravisdesell:mainfrom
Swaraj26:main
Open

committing sho#22
Swaraj26 wants to merge 1 commit intotravisdesell:mainfrom
Swaraj26:main

Conversation

@Swaraj26
Copy link

@Swaraj26 Swaraj26 commented Feb 7, 2023

Made SHO changes for your review, commented //SHO SY wherever changes have been made.

@jkarns275
Copy link
Collaborator

It looks like you may not have merged with the main branch in a while, I think you'll have to do that before we can proceed. If merging turns out to be a big headache you're probably going to have to run the formatter on your branch before you attempt to merge.

If you're on a mac you can install clang-format using brew install clang-format, ubuntu has a clang-format packages as well you can install with apt.

Once you have that installed just run the format script inside the exact directory:

~/exact $ sh scripts/util/format.sh

This will automatically format your code and you can just commit the changes, hopefully without much trouble :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants